Skip to content

feat(voice): apply Pocket speed changes during active speech - #208

Merged
johnmatthewtennant merged 6 commits into
mainfrom
jtennant/live-pocket-playback-speed
Aug 26, 2026
Merged

feat(voice): apply Pocket speed changes during active speech#208
johnmatthewtennant merged 6 commits into
mainfrom
jtennant/live-pocket-playback-speed

Conversation

@johnmatthewtennant

Copy link
Copy Markdown
Contributor

Summary

Pocket playback speed was previously captured when an utterance started, so changing the setting affected only later speech. This makes speed changes apply to the current preview, one-shot utterance, or streaming response without restarting playback. The native audio graph remains stable through 1x transitions, and spoken-delivery progress stays conservative and monotonic as the rate changes.

Reviewer-reproducible examples

  1. Select Pocket for speech output and start a voice conversation with a multi-paragraph response.
  2. While the response is speaking, change playback speed from 1x to 2x, then back to 1x.
  3. Confirm the current utterance changes speed without restarting, dropping queued speech, or changing pitch.

@johnmatthewtennant
johnmatthewtennant force-pushed the jtennant/live-pocket-playback-speed branch 2 times, most recently from 1a65a0a to 78de2b2 Compare August 25, 2026 16:53
Base automatically changed from jtennant/native-pocket-playback-speed to main August 26, 2026 00:16
@johnmatthewtennant
johnmatthewtennant force-pushed the jtennant/live-pocket-playback-speed branch from 78de2b2 to ceb584e Compare August 26, 2026 01:02
@johnmatthewtennant
johnmatthewtennant marked this pull request as ready for review August 26, 2026 02:13
@johnmatthewtennant
johnmatthewtennant requested a review from a team August 26, 2026 02:13

@morgmart morgmart left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 Automated code review

APPROVE. Fresh static review of the exact three-dot comparison found no publishable findings. Final self-check covered the changed live-speed flows for preview, one-shot speech, and streaming speech; accessibility and localization impact; navigation and consent guards; async errors, never-completes behavior, lifecycle and races; test honesty; project and design-system rules; duplicate overlap; and the evidence required for blocking severity. The supplied GitHub evidence reports all eight check runs completed successfully, while the legacy combined commit status is pending with no status contexts; required checks still independently govern merge readiness.

Deterministic publication result: 0 blocking and 0 non-blocking finding(s) publishable; 0 duplicate(s) suppressed.

Pending checks: 1 check(s) are not complete.

This approval reflects the completed code review only; merge readiness remains governed by the repository's required checks.

@johnmatthewtennant
johnmatthewtennant merged commit 658a1c8 into main Aug 26, 2026
8 checks passed
@johnmatthewtennant
johnmatthewtennant deleted the jtennant/live-pocket-playback-speed branch August 26, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants